-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
USWDS-Site: Updating links per Design Principles audit #3053
base: main
Are you sure you want to change the base?
Conversation
After extensive audit of links to determine relevance of links, edits can be made to this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finished my review and added my comments to the page draft since it's probably clearer there. There was one link in the draft that I didn't see in the preview (18F User Experience Guide — Research ), and I commented on 3 other links - one that I think we decided should be removed, one that was duplicative, and one where I wondered if there was a more up-to-date resource to link to.
Made a few corrections per comments from JC in the draft doc.
@jaclinec - I made updates per your comments in the draft. Please review when you are up to it. |
@finekatie everything looks good now - thanks! |
I reviewed links to make sure nothing was broken due to recent policy changes. All looks good. @annepetersen and @thisisdano this is ready for FFR. |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I read over and clicked all the links focusing on the a11y section. The only slight oddity is that when I click the plain language link I see a bunch of weird code (maybe tracking code?) in the URL
1.Try clicking the "learn about writing in plain language" link under "further reading" in the "embrace accessibility" section.
- Then look at your browser bar to see the URL or see below:
It didn't seem to do that on the other plain language link on the page. I also deleted that code and the URL works fine. So maybe just a weird glitch (?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just reviewed it and all looks good except 2 links at the end of the page, which are broken:
Summary
After extensive audit of links to determine relevance of links, edits can be made to this page.
Important
This page does not have a changelog - one needs to be added
Related issue
Closes #3054 / #2573
Preview link
Design principles page
Problem statement
Resources linked throughout the Design Principles page were aging or no longer relevant.
Solution
UX, CS, A11y teams and feds to audit and analyze relevance and timeliness of links, update and add new as needed.
Related documents:
Audit
Page draft with edits
Testing and review
Need reviews to compare to the page draft make sure links work and content is clean and clear.